Skip to content

manifest: fix missing GRUB console.cfg in bootc disk images - #2829

Open
Roshan-R wants to merge 1 commit into
osbuild:mainfrom
Roshan-R:fix/grub-mount
Open

Roshan-R wants to merge 1 commit into
osbuild:mainfrom
Roshan-R:fix/grub-mount

Conversation

@Roshan-R

Copy link
Copy Markdown

The grub2.d stage mounts the image filesystems but has no bind mount
connecting them to the pipeline tree. Writing console.cfg through
tree:// therefore leaves the file outside the generated disk image.

Change the destination to mount://boot/grub2/console.cfg and update
the expected paths in the serialization tests.

@Roshan-R
Roshan-R requested a review from a team as a code owner November 12, 2026 11:53
@Roshan-R

Copy link
Copy Markdown
Author

The issue was originally filed in joelcapitao/bib-fcos-experimentation#166

@supakeen

Copy link
Copy Markdown
Member

cc @jbtrystram / @joelcapitao as I seem to recall either of you working on this or at least mentioning it before?

@supakeen

Copy link
Copy Markdown
Member

Ah I see @jbtrystram already commented on this in the linked issue.

@supakeen supakeen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense.

@Roshan-R

Roshan-R commented Nov 13, 2026 •

Copy link
Copy Markdown
Author

@supakeen thanks for approving the PR!
CI is blocking the merge but I can't re-trigger it

@supakeen

Copy link
Copy Markdown
Member

@supakeen thanks for approving the PR!

CI is blocking the merge but I can't re-trigger it

I've triggered them.

The grub2.d stage mounts the image filesystems but has no bind mount
connecting them to the pipeline tree. Writing console.cfg through
tree:// therefore leaves the file outside the generated disk image.

Change the destination to mount://boot/grub2/console.cfg and update
the expected paths in the serialization tests.
@Roshan-R

Copy link
Copy Markdown
Author

@supakeen I had to rebase the branch to merge and it looks like we have to run CI again.
could you trigger them once again please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants